-
Notifications
You must be signed in to change notification settings - Fork 19.6k
[OpenVINO backend] Remove usage of deprecated openvino.runtime
#21418
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #21418 +/- ##
==========================================
- Coverage 82.76% 82.76% -0.01%
==========================================
Files 565 565
Lines 55378 55375 -3
Branches 8646 8646
==========================================
- Hits 45833 45830 -3
Misses 7442 7442
Partials 2103 2103
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This reverts commit de0fb67.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do NOT merge, openvino test is failing
@fchollet @mattdangerw The CI is failing with pre-commit check diff --git a/keras/api/optimizers/__init__.py b/keras/api/optimizers/__init__.py
index 40f6ab401..4114fab5b 100644
--- a/keras/api/optimizers/__init__.py
+++ b/keras/api/optimizers/__init__.py
@@ -4,6 +4,7 @@ This file was autogenerated. Do not edit it by hand,
since your modifications would be overwritten.
"""
+
from keras.optimizers import legacy as legacy
from keras.optimizers import schedules as schedules
from keras.src.optimizers import deserialize as deserialize
@@ -18,9 +19,7 @@ from keras.src.optimizers.adamw import AdamW as AdamW
from keras.src.optimizers.ftrl import Ftrl as Ftrl
from keras.src.optimizers.lamb import Lamb as Lamb
from keras.src.optimizers.lion import Lion as Lion
-from keras.src.optimizers.loss_scale_optimizer import (
- LossScaleOptimizer as LossScaleOptimizer,
-)
+from keras.src.optimizers.loss_scale_optimizer import LossScaleOptimizer as LossScaleOptimizer
from keras.src.optimizers.muon import Muon as Muon Is this expected? @rkazants suggested that it may be caused by wrong ordering of imports in my PR, but I changed it in numerous different ways and the check still fails. |
openvino.runtime
is deprecated in favor of flatopenvino
and is set to be removed in the upcoming OpenVINO release.